Skip to content
This repository was archived by the owner on Sep 11, 2024. It is now read-only.

Live location sharing: create beacon info event from location picker #8072

Merged
merged 5 commits into from
Mar 18, 2022

Conversation

kerryarchibald
Copy link
Contributor

@kerryarchibald kerryarchibald commented Mar 17, 2022

With: matrix-org/matrix-js-sdk#2245

Creates a beacon_info event with a default 5min timeout

Screen.Recording.2022-03-17.at.13.11.37.mov

Done:

  • Send m.beacon_info.* state events.
  • track m.beacon_info.* events in room state
  • expose and maintain 'user room has live beacons' property in room state
  • add ability to monitor liveness of beacon
  • get users own beacons from room state, monitor liveness
  • maintain 'user has live beacons' property)
  • show live state

In this PR:

  • create beacons from the location picker

In next PRs:

  • produce good unique suffixes for beacon info events
  • geolocate and send m.beacon events for live beacons
  • maintain liveness state of beacons (checking for expiry, flipping isLive on event content)

This PR currently has no changelog labels, so will not be included in changelogs.

A reviewer can add one of: T-Deprecation, T-Enhancement, T-Defect, T-Task to indicate what type of change this is, or add Type: [enhancement/defect/task] to the description and I'll add them for you.

Preview: https://pr8072--matrix-react-sdk.netlify.app
⚠️ Do you trust the author of this PR? Maybe this build will steal your keys or give you malware. Exercise caution. Use test accounts.

@kerryarchibald kerryarchibald requested a review from a team as a code owner March 17, 2022 12:13
Signed-off-by: Kerry Archibald <[email protected]>
@codecov
Copy link

codecov bot commented Mar 17, 2022

Codecov Report

Merging #8072 (d535840) into develop (176e49e) will increase coverage by 0.02%.
The diff coverage is 83.33%.

@@             Coverage Diff             @@
##           develop    #8072      +/-   ##
===========================================
+ Coverage    27.12%   27.14%   +0.02%     
===========================================
  Files          868      868              
  Lines        52097    52113      +16     
  Branches     13199    13204       +5     
===========================================
+ Hits         14130    14148      +18     
+ Misses       37967    37965       -2     
Impacted Files Coverage Δ
src/components/views/location/shareLocation.ts 84.84% <78.26%> (+18.18%) ⬆️
src/components/views/location/LocationPicker.tsx 92.55% <100.00%> (ø)
...rc/components/views/location/LocationShareMenu.tsx 100.00% <100.00%> (ø)

@kerryarchibald kerryarchibald merged commit cdcf6d0 into develop Mar 18, 2022
@kerryarchibald kerryarchibald deleted the psf-662/send-beacon-info-event branch March 18, 2022 09:52
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants